fix more tests with regex

fix test_cargo_test

fix some tests with IntelliJ-Rust automated refactoring

fix some more tests with regexp

fix a bunch of tests with regexp

fix test_cargo_clean

fix test_cargo_build_lib

fix test_cargo_build_auth

report status to stderr instead of stdout

Auto merge of #2703 - alexcrichton:target-dir-abs-path, r=alexcrichton

Fix relative `build.target-dir` configuration

Closes #2700

Auto merge of #2706 - alx741:extend_manpages, r=alexcrichton

Add man pages for cargo commands

So, the markdown port (#2672) is in idle, I'll be working in a way to make it without depending on external tools, but that is going to take a bit longer.

These are the **troff** pages for the rest of the cargo commands; previously the *cargo-build(1)* was added, so here are the rest.

No extra build dependencies here, just the new man pages.

Auto merge of #2688 - carols10cents:docs-refresh, r=alexcrichton

Docs refresh

Hi! Sooooo I've been thinking about the cargo docs lately and today I decided to try some of the things I've been thinking about.

There are some smaller changes in here, typos and clarifications and stuff, that I'm totally willing to break out into a separate PR(s) if yinz want.

My goals for the larger changes were:

- **Make one obvious place to go for info on every which way to specify a dependency.** I've found myself clicking around the different doc pages a lot to find the semver syntax, the different keys that can go with a git location, the `path` override syntax, etc. So I made one top-level "Specifying Dependencies" page and moved relevant info from the guide, the manifest page, and the crates.io page.
- **Make The Guide a bit more focused.** There were a few topics in there that didn't really fit in the story arc of trying to get someone up and running with cargo-- it veered off into some more advanced topics that I've moved to places I think they fit better.
- **Make the crates.io page only about publishing, not using and publishing.** The info about adding dependencies got folded into either the guide or the new specifying dependencies page, so this should close https://github.com/rust-lang/cargo/issues/1035.

I'm also thinking about having someone new to rust/cargo try following the guide, but wanted to get a first pass from experienced people :)

Auto merge of #2713 - sbeckeriv:fix-name-ordering, r=alexcrichton

Correct author order

Dearest Reviewer,

PR #2696 added some new checks for environment variables for author and
email. After the pull request was merged a comment was left about gits
ordering for looking at the variables. This pull request closes #2705 by
reordering the look up and also addeds CARGO_NAME and CARGO_EMAIL to the
top of the order.

Thanks
Becker

Correct author order

Dearest Reviewer,

PR #2696 added some new checks for environment variables for author and
email. After the pull request was merged a comment was left about gits
ordering for looking at the variables. This pull request closes #2705 by
reordering the look up and also addeds CARGO_NAME and CARGO_EMAIL to the
top of the order.

Thanks
Becker

At the end, point to docs that might be interesting next

Add info about what's different between debug/release

Add info about `build-dependencies` to the specifying deps page

Add a `[dev-dependencies]` example

Correct typo when referring to 64 bit

Auto merge of #2707 - sbeckeriv:banned-list-2699, r=alexcrichton

Ban keywords from crate names

Dearest Reviewer,

This pull request extends the banned list of project names to include
all of the current keywords for rust. I got the list of keywords from
https://doc.rust-lang.org/grammar.html#keywords . This
closes #2699 . The original ticket said warn but I figured how test is
handled is most likely how all banned names should be handled.

Oddly enough a few keywords have made their way to the crates.io. fn and
proc are both examples I found spot checking keywords. I do not there is
any action to take on those crates.

Thanks
Becker

Ban keywords from crate names

Dearest Reviewer,

This pull request extends the banned list of project names to include
all of the current keywords for rust. I got the list of keywords from
https://doc.rust-lang.org/grammar.html#keywords . This
closes #2699 . The original ticket said warn but I figured how test is
handled is most likely how all banned names should be handled.

Oddly enough a few keywords have made their way to the crates.io. fn and
proc are both examples I found spot checking keywords. I do not there is
any action to take on those crates.

Thanks
Becker

Auto merge of #2685 - mkroman:master, r=alexcrichton

Fix `cargo test --doc` not testing examples

When running cargo doc --test in a project folder, cargo does not test documentation examples, it only builds the project and exits.

Refer to issue #2684

Revert "Maybe fix failing build on OS X rust beta"

This reverts commit 3fb3fdc82fdf3a1ea060d51076c86c71fe1f405b.

Man pages: add cargo-search

Man pages: add cargo-publish

Man pages: add cargo-package

Man pages: add cargo-update

Man pages: add cargo-install

Man pages: add cargo-bench

Man pages: fix cargo-test

Maybe fix failing build on OS X rust beta

Man pages: add cargo-test

Man pages: remove options from cargo-run

Add cargo-init reference in cargo-new

Man pages: add cargo-run

Man pages: add cargo-init

Man pages: add cargo-new

Man pages: add cargo-doc

Man pages: add cargo-clean

Man pages: Improve cargo-build description

Auto merge of #2696 - sbeckeriv:an-author-by-another-name-1213, r=alexcrichton

An author by another name

Dearest Reviewer

This is to support environment variables for name and email more like
git. This closes #1213 . I did look in to using libgit2 but I did not
see a clear way to get the author ident. This just moves from or_else
chaining to map_filter with env checks and getting the nth(0) result.

The docs I found
https://git-scm.com/book/en/v2/Git-Internals-Environment-Variables#Committing
do not really indicate an order of preference to the variables. I am
more then happy to rearrange the arrays in anyway.

I also did not add in a new test for the positive case because the
test for the current variables cover that path.

Thanks for reviewing,
Becker

An author by another name

Dearest Reviewer

This is to support environment variables for name and email more like
git. This closes #1213 . I did look in to using libgit2 but I did not
see a clear way to get the author ident. This just moves from or_else
chaining to map_filter with env checks and getting the nth(0) result.

The docs I found
https://git-scm.com/book/en/v2/Git-Internals-Environment-Variables#Committing
do not really indicate an order of preference to the variables. I am
more then happy to rearrange the arrays in anyway.

Thanks for reviewing,
Becker

Use dedicated directory for man pages

Auto merge of #2701 - alexcrichton:target-no-dylib, r=brson

Return a normal error if dylibs aren't supported

Rebasing of https://github.com/rust-lang/cargo/pull/2437 with a few comments addressed

Closes https://github.com/rust-lang/cargo/issues/2427

Add missing newline in test output

Change `cargo test --doc` test to match stdout

Fix relative `build.target-dir` configuration

Closes #2700

Touch up the error message and warnings

Merge branch 'issue-2427' of https://github.com/jespino/cargo into target-no-dylib

Clarify statements about ignoring/checking in Cargo.lock

Small rewordings

Clarify that you should not manually edit Cargo.lock

Add an ascii directory tree to the project layout section

Clarify that integration tests go in tests/

"External" is a bit confusing. Then if we say where integration tests
go, that might make someone wonder where the unit tests go, so I added a
note about unit tests.

Auto merge of #2691 - serprex:master, r=alexcrichton

Remove dependency on time in favor of std::time

Auto merge of #2690 - hatsunearu:hatsunearu-patch-1, r=alexcrichton

Update manpage with some missing cargo commands

the manpage of cargo is missing a few important cargo commands, and I've added `rustc` and `help`. Hopefully everything's alright.

Auto merge of #2676 - srinivasreddy:refactor, r=alexcrichton

remove sleep_ms and change it to sleep

Reword note about git init default to be in the positive

Remove section about converting a project to cargo

Since most projects start with cargo these days.

Remove dependency on time in favor of std::time

Replace references to conduit(-static) with uuid/rand

Since they're official rust crates.

Combine related thoughts into the same paragraph

Reword to not call a non-rust-tuple thing a tuple

Use — instead of --

Use code formatting for `git`

Add all three channels to the sample .travis.yml

Replace refs to color-rs crate to rust-lang's rand crate

Bonus: rand has a `next` branch so the git branch example makes more
sense!

Clarify that release builds go in the release dir, not another subdir

Good call @steveklabnik! This matches much more nicely with the contrast
to `target/debug`.

Link crates.io everywhere

Fix markdown links that repeat their link text unnecessarily

Update manpage with some missing cargo commands

Add instructions for rendering docs locally to the README

Make the crates.io docs about publishing only, not using

Try to make this flow a bit more and have sections based on what
situation you might find yourself in.

Consolidate all info about specifying deps into one page.

I think information about specifying dependencies deserves to have its
own page since this info is commonly needed and there are a LOT of
different ways to specify dependencies.

Previously this information was spread across multiple pages, now
there's one place to go if you're looking for any of these ways, rather
than checking the multiple pages this info was spread across before.

Incorporate "using crates from crates.io" into the main guide

Connects to rust-lang/cargo#1035. Using crates from crates.io is a
common thing that many people will want to know how to do right away and
deserves to be part of the main guide.

Move the Travis CI section under the testing section

I'm guessing the path dependency section got added in between. Also
linked to the Travis CI rust docs and removed the hyphen since Travis CI
seems to spell it without.

Small rewording changes; mostly to clear up potential ambiguity

Correct reference to `foo` to `hello_world`

Copypasta, or forgotten name change

Clarify that the "somewhere" is github

Mention that binary location changes when using --release

Remove out of place instructions on passing args and flags

This doesn't feel like it fits very well at this point, since our
example program doesn't take args and beginners to cargo probably don't
have a reason to want to pass flags on to rustc at this point. Also it
doesn't even say rustc, just "which flags go where"... where would they
go?!?!

Correct `Fresh` to `Compiling` since we aren't running with -v

I only see `Fresh` if I use `cargo run -v`, but this example uses `cargo
run`. Someone following along might not see either of these if they just
built in the previous step, though, so clarify that you'll only see that
if you have made changes.

Remove extranneous commas

Steve and commas, sittin in a tree!

Fixes rust-lang/crates.io#334.

Auto merge of #2682 - srinivasreddy:refactor_2, r=alexcrichton

remove deprecated attr - change connect() to join()

Fix `cargo test --doc` not testing examples

remove deprecated attr - change connect() to join()

Auto merge of #2681 - alexcrichton:cargo-env-home, r=brson

Canonicalize CARGO_HOME fallback on Windows

This commit ensures that we always return the same fallback value on Windows
regardless of whichever shell we happen to be run from. We do this by removing
the `$HOME` environment variable which `std::env::home_dir` will inspect to
force it to fall back to the system APIs. If the old directory exists then we
favor that one, but otherwise we favor locations like `C:\Users\$user`

Supercedes and closes #2604

Canonicalize CARGO_HOME fallback on Windows

This commit ensures that we always return the same fallback value on Windows
regardless of whichever shell we happen to be run from. We do this by removing
the `$HOME` environment variable which `std::env::home_dir` will inspect to
force it to fall back to the system APIs. If the old directory exists then we
favor that one, but otherwise we favor locations like `C:\Users\$user`

Supercedes and closes #2604

remove deprecated function sleep_ms and change it to sleep(Duration::from_millis..)

Auto merge of #2678 - matklad:simplify-tests-2, r=alexcrichton

Simplify more tests

This is the followup of #2670

It contains [one](https://github.com/rust-lang/cargo/commit/28887beee054bfa9b56207667df3f5ae38a8ac6f#diff-ebcf8bfb935037902f135a7c225479b5L322) more or less significant change, everything else is just search/replace.

r? @alexcrichton

remove useless formats

simplify tests

remove ADDING, REMOVING, DOCTEST, DOWNLOADING, PACKAGING, UPLOADING,
VERIFYING, ARCHIVING, INSTALLING, REPLACING constants

Auto merge of #2396 - sbeckeriv:network-retry-1602, r=alexcrichton

Network retry issue 1602

Dearest Reviewer,

This branch resolves #1602 which relates to retrying network
issues automatically. There is a new utility helper for retrying
any network call.

There is a new config called net.retry value in the .cargo.config
file. The default value is 2. The documentation has also been
updated to reflect the new value.

Thanks
Becker

Network retry issue 1602

Dearest Reviewer,

This branch resolves #1602 which relates to retrying network
issues automatically. There is a new utility helper for retrying
any network call.

There is a new config called net.retry value in the .cargo.config
file. The default value is 2. The documentation has also been
updated to reflect the new value.

Thanks
Becker

Auto merge of #2674 - alexcrichton:less-third-party-cruft, r=brson

Don't print extra error info for subcommands

Assume they take care of error printing, so just ferry along the exit status if
they fail

Closes #2673

Don't print extra error info for subcommands

Assume they take care of error printing, so just ferry along the exit status if
they fail

Closes #2673

Auto merge of #2670 - matklad:simplify-tests, r=alexcrichton

Simplify tests

@alexcrichton I think there is one bit of tests that could be simplified.

What do you think about writing this

```Rust
test!(simple {
    pkg("foo", "0.0.1");

    assert_that(cargo_process("install").arg("foo"),
                execs().with_status(0).with_stdout(&format!("\
[UPDATING] registry `[..]`
[DOWNLOADING] foo v0.0.1 (registry file://[..])
[COMPILING] foo v0.0.1 (registry file://[..])
[INSTALLING] {home}[..]bin[..]foo[..]
",
        home = cargo_home().display())));
    assert_that(cargo_home(), has_installed_exe("foo"));

    assert_that(cargo_process("uninstall").arg("foo"),
                execs().with_status(0).with_stdout(&format!("\
[REMOVING] {home}[..]bin[..]foo[..]
",
        home = cargo_home().display())));
    assert_that(cargo_home(), is_not(has_installed_exe("foo")));
});
```

instead of this

```Rust
test!(simple {
    pkg("foo", "0.0.1");

    assert_that(cargo_process("install").arg("foo"),
                execs().with_status(0).with_stdout(&format!("\
{updating} registry `[..]`
{downloading} foo v0.0.1 (registry file://[..])
{compiling} foo v0.0.1 (registry file://[..])
{installing} {home}[..]bin[..]foo[..]
",
        updating = UPDATING,
        downloading = DOWNLOADING,
        compiling = COMPILING,
        installing = INSTALLING,
        home = cargo_home().display())));
    assert_that(cargo_home(), has_installed_exe("foo"));

    assert_that(cargo_process("uninstall").arg("foo"),
                execs().with_status(0).with_stdout(&format!("\
{removing} {home}[..]bin[..]foo[..]
",
        removing = REMOVING,
        home = cargo_home().display())));
    assert_that(cargo_home(), is_not(has_installed_exe("foo")));
});
```

This PR has a proof of concept implementation of this feature applied to a couple of tests.

r? @alexcrichton

simplify tests

remove RUNNING, COMPILING, ERROR, DOCUMENTING, FRESH and UPDATING
constants

Auto merge of #2669 - mbrubeck:dev-dep-doc, r=alexcrichton

Document that dev-dependencies are used when building examples

None

Simplify tests